Skip to content

Fix Parquet queryable error handling#7638

Merged
SungJin1212 merged 2 commits into
cortexproject:masterfrom
SungJin1212:fix/parquet-querier-error-return
Jun 29, 2026
Merged

Fix Parquet queryable error handling#7638
SungJin1212 merged 2 commits into
cortexproject:masterfrom
SungJin1212:fix/parquet-querier-error-return

Conversation

@SungJin1212

@SungJin1212 SungJin1212 commented Jun 23, 2026

Copy link
Copy Markdown
Member

What this PR does:
This PR fixes error handling in parquet queryable.

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]
  • docs/configuration/v1-guarantees.md updated if this PR introduces experimental flags

@SungJin1212 SungJin1212 changed the title Fix Parquet error handling Fix Parquet queryable error handling Jun 23, 2026
@SungJin1212 SungJin1212 force-pushed the fix/parquet-querier-error-return branch from 9497f65 to c30c809 Compare June 24, 2026 02:18
Signed-off-by: SungJin1212 <tjdwls1201@gmail.com>
Signed-off-by: SungJin1212 <tjdwls1201@gmail.com>
@SungJin1212 SungJin1212 force-pushed the fix/parquet-querier-error-return branch from c30c809 to dc8322b Compare June 26, 2026 01:58

@friedrichg friedrichg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Jun 28, 2026
@SungJin1212 SungJin1212 merged commit c15df9b into cortexproject:master Jun 29, 2026
40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/querier lgtm This PR has been approved by a maintainer size/XS type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants